Skip to content

docs: inline ndarray output in doctests and add missing article#11557

Merged
kgryte merged 5 commits intodevelopfrom
philipp/fix-propagation-2026-04-18
Apr 19, 2026
Merged

docs: inline ndarray output in doctests and add missing article#11557
kgryte merged 5 commits intodevelopfrom
philipp/fix-propagation-2026-04-18

Conversation

@Planeshifter
Copy link
Copy Markdown
Member

@Planeshifter Planeshifter commented Apr 18, 2026

Description

Propagates two fixes from develop (window: 3af178cc0414d61b8a) to sibling packages with the same defect.

7be0aa66f et al. — inline ndarray output in doctests. Drops the @stdlib/ndarray/to-array require and trailing var arr = ndarray2array( v ) block; folds the value into // returns <ndarray>[...]. Applied to stats/base/ndarray/{scumaxabs,scuminabs}, blas/ext/base/ndarray/{dcusumkbn,dsortins,dsortsh,gcusumkbn,gcusumkbn2,gcusumors,gcusumpw,glinspace,gsorthp,scusumkbn,scusumkbn2,scusumors,slinspace,ssorthp}, and the two namespace index.d.ts files (10 + 13 @example blocks). Runnable ## Examples blocks left untouched.

d4c1b8b1a / b8ad692ed — missing article in *axpy test description. "one-dimensional ndarray x" → "a one-dimensional ndarray x" (and same for y) in blas/base/ndarray/{gaxpy,saxpy,daxpy}/test/test.js.

Related Issues

No.

Questions

No.

Other

Excluded: 850600f45 (BigInt note in napi/argv-*) and 3af178cc0 (backtick requires) — both already fully propagated; namespace TOC / related-packages updates — not generalizable.

Checklist

AI Assistance

When authoring the changes proposed in this PR, did you use any kind of AI assistance?

  • Yes
  • No

If you answered "yes" above, how did you use AI assistance?

  • Code generation (e.g., when writing an implementation or fixing a bug)
  • Test/benchmark generation
  • Documentation (including examples)
  • Research and understanding

Disclosure

Authored by Claude Code as an automated fix-propagation routine.


@stdlib-js/reviewers

claude added 2 commits April 18, 2026 23:15
…escriptions

Propagates fix from d4c1b8b1a ("test: update description") and b8ad692
("test: update description") to sibling axpy packages — inserting missing
"a " before "one-dimensional ndarray `x`" and "one-dimensional ndarray `y`"
in the top-level tape() description. Affects `gaxpy`, `saxpy`, `daxpy`.

https://claude.ai/code/session_01GXZT2uLjX36Q2EepAt4GeQ
… and `blas/ext/base/ndarray/*`

Propagates fix from 7be0aa6, a6b67f6, 71a93c8, adc6eb6, d5dc0ec,
9e562a3, a19ed4c, 00385f6 ("docs: improve doctests for ndarray
instances in ..."), and 849495e ("docs: update example") to sibling
ndarray-instance packages — removing the `ndarray2array` import plus the
trailing `var arr = ndarray2array( v )` block and inlining the array value
into the `// returns <ndarray>[...]` comment. Applied to:

- stats/base/ndarray: scumaxabs, scuminabs, plus 10 `@example` blocks in
  stats/base/ndarray/docs/types/index.d.ts (cumax, cumin, dcumax, dcumaxabs,
  dcumin, dcuminabs, scumax, scumaxabs, scumin, scuminabs).
- blas/ext/base/ndarray: dcusumkbn (main.js only), dsortins, dsortsh,
  gcusumkbn, gcusumkbn2, gcusumors, gcusumpw, glinspace, gsorthp, scusumkbn,
  scusumkbn2, scusumors, slinspace, ssorthp, plus 13 `@example` blocks in
  blas/ext/base/ndarray/docs/types/index.d.ts (dsortins, dsortsh, gcusumkbn,
  gcusumkbn2, gcusumors, gcusumpw, glinspace, gsorthp, scusumkbn, scusumkbn2,
  scusumors, slinspace, ssorthp).

The `Examples` section of each README (runnable `console.log` blocks) is
intentionally untouched; the fix only applies to the Usage doctest blocks.

https://claude.ai/code/session_01GXZT2uLjX36Q2EepAt4GeQ
@stdlib-bot
Copy link
Copy Markdown
Contributor

stdlib-bot commented Apr 18, 2026

Coverage Report

Package Statements Branches Functions Lines
blas/base/ndarray/daxpy $\color{green}123/123$
$\color{green}+0.00%$
$\color{green}3/3$
$\color{green}+0.00%$
$\color{green}1/1$
$\color{green}+0.00%$
$\color{green}123/123$
$\color{green}+0.00%$
blas/base/ndarray/gaxpy $\color{green}121/121$
$\color{green}+0.00%$
$\color{green}3/3$
$\color{green}+0.00%$
$\color{green}1/1$
$\color{green}+0.00%$
$\color{green}121/121$
$\color{green}+0.00%$
blas/base/ndarray/saxpy $\color{green}123/123$
$\color{green}+0.00%$
$\color{green}3/3$
$\color{green}+0.00%$
$\color{green}1/1$
$\color{green}+0.00%$
$\color{green}123/123$
$\color{green}+0.00%$
blas/ext/base/ndarray/dcusumkbn $\color{green}123/123$
$\color{green}+0.00%$
$\color{green}3/3$
$\color{green}+0.00%$
$\color{green}1/1$
$\color{green}+0.00%$
$\color{green}123/123$
$\color{green}+0.00%$
blas/ext/base/ndarray $\color{red}9771/10449$
$\color{green}+0.00%$
$\color{red}171/177$
$\color{green}+0.00%$
$\color{red}0/88$
$\color{green}+0.00%$
$\color{red}9771/10449$
$\color{green}+0.00%$
stats/base/ndarray $\color{red}21874/23184$
$\color{red}-0.01%$
$\color{red}382/403$
$\color{green}+0.06%$
$\color{red}0/201$
$\color{green}+0.00%$
$\color{red}21874/23184$
$\color{red}-0.01%$

The above coverage report was generated for the changes in this PR.

@Planeshifter Planeshifter changed the title Propagate recent docs / test fixes across sibling ndarray packages docs: inline ndarray output in doctests across stats/base/ndarray/* and blas/ext/base/ndarray/* Apr 18, 2026
@Planeshifter Planeshifter changed the title docs: inline ndarray output in doctests across stats/base/ndarray/* and blas/ext/base/ndarray/* docs: inline ndarray output in doctests across sibling ndarray packages Apr 18, 2026
@kgryte kgryte added the bot: Merge Pull request needing the latest project code be merged in. label Apr 19, 2026
@stdlib-bot stdlib-bot added bot: In Progress Pull request is currently awaiting automation. and removed bot: Merge Pull request needing the latest project code be merged in. labels Apr 19, 2026
@stdlib-js stdlib-js deleted a comment from stdlib-bot Apr 19, 2026
@Planeshifter Planeshifter marked this pull request as ready for review April 19, 2026 05:01
@Planeshifter Planeshifter requested a review from a team April 19, 2026 05:01
@stdlib-bot stdlib-bot added the Needs Review A pull request which needs code review. label Apr 19, 2026
@Planeshifter Planeshifter changed the title docs: inline ndarray output in doctests across sibling ndarray packages docs: inline ndarray output in doctests and add missing article Apr 19, 2026
@Planeshifter Planeshifter requested a review from kgryte April 19, 2026 05:02
@kgryte kgryte removed the bot: In Progress Pull request is currently awaiting automation. label Apr 19, 2026
@kgryte
Copy link
Copy Markdown
Member

kgryte commented Apr 19, 2026

/stdlib merge

@stdlib-bot stdlib-bot added the bot: In Progress Pull request is currently awaiting automation. label Apr 19, 2026
@stdlib-bot stdlib-bot removed the bot: In Progress Pull request is currently awaiting automation. label Apr 19, 2026
@kgryte
Copy link
Copy Markdown
Member

kgryte commented Apr 19, 2026

/stdlib merge

@stdlib-bot stdlib-bot added the bot: In Progress Pull request is currently awaiting automation. label Apr 19, 2026
@stdlib-bot stdlib-bot removed the bot: In Progress Pull request is currently awaiting automation. label Apr 19, 2026
@kgryte kgryte removed the Needs Review A pull request which needs code review. label Apr 19, 2026
@kgryte
Copy link
Copy Markdown
Member

kgryte commented Apr 19, 2026

As the CI failure is not due to changes introduced in this PR, I'll go ahead and merge.

@kgryte kgryte merged commit 0873811 into develop Apr 19, 2026
14 of 16 checks passed
@kgryte kgryte deleted the philipp/fix-propagation-2026-04-18 branch April 19, 2026 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants